-
Notifications
You must be signed in to change notification settings - Fork 431
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix vulnerability caused by filename escaping #877
Conversation
Hey thanks so much for the fix! I will take a look on this tonight. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small request, otherwise this looks good 👍
@kittinunf Thank you for your review 🙇 |
@motoyasu-saburi I will release the patch for this soon please hang on tight. |
@kittinunf thank you for the library. When are you planing to push the v2 release? |
Detail:
#876
Local tests have not yet been run.(I need to set up the Android-SDK perimeter, which is taking a bit of time.)